New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore: do not clone already recovered block #13944

Merged

mattsse merged 1 commit into main from dan/reuse-block

Jan 23, 2025

+1 −4

Member

Rjected commented Jan 23, 2025

We already have the recovered block, so we can just Arc it instead of cloning the senders and the sealed block


          chore: do not clone already recovered block

6d96be3

Rjected added C-enhancement A-blockchain-tree labels

Rjected requested review from rkrasiuk, mattsse and fgimenez as code owners

January 23, 2025 03:40

mattsse approved these changes

View reviewed changes

Collaborator

mattsse left a comment

I suspect we have a few more of those since the recent refactor

mattsse added this pull request to the merge queue

Merged via the queue into main with commit 5a40c68

44 checks passed

mattsse deleted the dan/reuse-block branch

January 23, 2025 16:18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-blockchain-tree C-enhancement